debezium/dbz#1725 fix: Improve loading state-management#380
Open
gmarav05 wants to merge 1 commit into
Open
Conversation
|
Hi @gmarav05, thanks for your contribution. Please prefix the commit message(s) with the debezium/dbz#xxx GitHub issue key. |
gmarav05
changed the title
gmarav05
force-pushed
the
fix/improve-loading-state-management
branch
2 times, most recently
from
fcce6ca to
ee41beb
Compare
Contributor
Author
gmarav05
force-pushed
the
fix/improve-loading-state-management
branch
from
ee41beb to
b41a6bd
Compare
Signed-off-by: Aravind <gmarav005@gmail.com>
gmarav05
force-pushed
the
fix/improve-loading-state-management
branch
from
b41a6bd to
c697c48
Compare
Member
|
@gmarav05 Are you able to record a video of the result? |
Contributor
Author
|
@mfvitale No, I did not record it. I will record it and post it here today. Thanks for letting me know. |
indraraj
requested changes
May 11, 2026
Member
indraraj
left a comment
indraraj
left a comment
There was a problem hiding this comment.
@gmarav05 Please rebase onto the latest main. Plus these are listing pages that you covered and have pooling, retry is not needed here instead limit polling after a few failed attempts and show an appropriate message to the user.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes debezium/dbz#1725
Description
This PR improves the initial loading state management in the Platform UI by addressing the following:
Suspensefallback inmain.tsx.QueryClientwithretry: 3. This ensures that in case of network issues, the UI will attempt to fetch data 3 times before displaying a network failure message, rather than showing a loading state indefinitely.Spinnerfor visual feedback.progressbarrole (associated with theSpinnercomponent) instead of searching for the now-removed text.PR Checklist
main